Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Updating the Tor binary included with Ricochet #618

Closed
wants to merge 3 commits into from
Closed

Updating the Tor binary included with Ricochet #618

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 25, 2018

Description

Screenshot: ricochet-symlinks

Resolves: #474

@ghost
Copy link

ghost commented Nov 25, 2018

Preview: http://62.102.148.154:19472/#im

@ghost
Copy link

ghost commented Nov 25, 2018

@beardog108 Thoughts?

@ghost ghost requested a review from Vincevrp November 25, 2018 15:07
@ghost
Copy link

ghost commented Nov 25, 2018

@tya99 I think it would be better if the ricochet warning was collapsible or linked to a github comment (like how we do the ipfs warning), because it takes up too much screen real estate just for people who want to use ricochet

Copy link
Contributor

@Vincevrp Vincevrp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tya99 I think it would be better if the ricochet warning was collapsible or linked to a github comment (like how we do the ipfs warning), because it takes up too much screen real estate just for people who want to use ricochet

I agree. Did you write this yourself @tya99? If not, please link to the original content instead.

@ghost
Copy link
Author

ghost commented Nov 26, 2018

I think it would be better if the ricochet warning was collapsible or linked to a github comment (like how we do the ipfs warning)

Could do that however I think that might detract from how important it is. Perhaps we could put it at the end after "Related Information".

Ideally it would be great if we had foldable headings on the privacytools.io page like this for example.

I agree. Did you write this yourself @tya99?

Yes I did. See Issue #474 part of the reason I am reluctant to just leave it as a github comment. Should something happen to github or the repository or even the privacytoolsIO account it would be lost and not tracked within source control.

@ghost
Copy link

ghost commented Nov 26, 2018

Books often have sections with info like this at the end. Maybe we could create a separate page for these things?

@ghost
Copy link

ghost commented Nov 26, 2018

@Shifterovich If we still have a link that would jump/go to the section, that would be acceptable.

@ghost
Copy link

ghost commented Nov 26, 2018

If we move this section to the bottom of the page, the links will work.

@ghost
Copy link
Author

ghost commented Jan 28, 2019

@privacytoolsIO
Copy link
Contributor

We'll figure something out soon.

privacytoolsIO pushed a commit that referenced this pull request Apr 2, 2019
privacytoolsIO pushed a commit that referenced this pull request Apr 2, 2019
privacytoolsIO pushed a commit that referenced this pull request Apr 2, 2019
@privacytoolsIO
Copy link
Contributor

Done. Thanks @tya99

@blacklight447 blacklight447 added the Tor Anything covering the Tor network label May 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Tor Anything covering the Tor network
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a warning to the Ricochet description
4 participants